Avoid deprecation warnings

We know regions are deprecated, no need to warn about it here.

Adwaita: set the correct indicator color for reorderable hover tabs

Adwaita: only side borders for reorderable tabs

Require GLib 2.41.1 (for G_PARAM_EXPLICIT_NOTIFY)

Adwaita: added active state to the switch slider

Adwaita: comment out button separators removal

...after reading #gnome-design scrollbark, since a consensus on how to achieve
this has not been reached yet.

Adwaita: fix view check/radio selected state.

Adwaita: disable separators in buttons

gdkwindow: Don't leak the surface in gdk_cairo_create()

Add a test for async icon loading

gtkmain: Rearrange code

Prevent a segfault in unknown circumstances, and break the ignore
deprecations wrapper out from the if statement so that we don't
cover too much ground by accident.

gtkwidget: Actually mark set_double_buffered as deprecated

Revert "gtkwidget: Widgets are now always double-buffered"

This reverts commit b875572f2af532b0e8f2dc368f8dc824b5f92724.

Apps like Abiword, gnumeric and gnome-chess, and toolkits like
ClutterGTK were all using this for various purposes, and this made them
break. Bring back this feature for now.

It still won't work under Wayland.

gdkwindow: Allow calling gdk_cairo_create outside of a "paint"

Apps had quite a bit of difficulty getting used to the new rules.
While we weren't expecting anything too deadly, it seemed that
gnumeric and Abiword both used gdk_cairo_create like this.

gdkwindow: Remove extra arguments from process_updates_recurse_helper

These can easily be calculated inside the function.

gdkwindow: Don't do some extra work

We can do an early return in some specific cases before we need to
create and clip a cairo region.

gdkwindow: Remove an unused variable

gdkwindow: Make the precondition of gdk_window_add_damage clearer

gdkwindow: Simplify check for damaging offscreen windows

If a window both has an impl and its impl_window is of type offscreen,
that must mean that it is the offscreen window, and the impl window is
itself. We can reduce the indirection here and make it more clear.

gdkwindow: Make gdk_window_add_damage more private

GtkIconTheme: Copy all fields in icon_info_dup

This functionality is only exercised by gnome-shell, currently.
Therefore, forgetting to copy a field here means an instant
gnome-shell crash :-(. More tests needed.

GtkIconTheme: Be more careful when warning

It sucks when printing a warning causes gnome-shell to crash,
so be more careful about icon names being NULL here.

GtkIconTheme: keep misplaced symbolics working

Recent changes made it a breaking mistake to install symbolic icons
of the wrong size into a theme directory, or into the legacy unthemed
icon location.

Since this change affects many apps, do the extra work to keep these
icons working, but emit warnings, in the hope that this will lead to
cleaning up the mess over time.

Move some comments around

Some documentation rewording

Adwaita: Use rtl variants of pan-end when needed

The -gtk-icontheme() function not have any automatism for
symbolic or rtl variants built-in, instead, we expect the theme
to use selectors to provide the right icon for each situation.

widget-factory: Fix another icon size problem

0 is still not a valid icon size.

win32: Drop remnants of def files

We no longer have .def files, so don't refer to them from the
win32 build machinery. Patch by Cilyan Olowen

placessidebar: Use symbolic name for symbolic icon

https://bugzilla.gnome.org/show_bug.cgi?id=731908

Docs: Reword deprecation notes

Mainly, make sure that deprecated properties are
documented as such. In this case: double-buffered.

notify test: Skip GtkWidget::double-buffered

This property is now deprecated, and ignored, thus: no notification.

widget-factory: Fix an icon size problem

0 is not a valid icon size, and setting it has unpredictable
results. Replace it by 1, which is the right icon size to use
in buttons.

GtkCellRendererPixbuf: Set force_scale_pixbuf

The recent icon theme scaling changes make the code more
sensitive to mis-sized icons (e.g. application icons in
the app chooser). A single row whose size gets blown out
of proportion by a big icon is never wanted in a list.
We can avoid this situation by telling GtkIconHelper to
force-scale the pixbuf to the requested size.

GtkIconHelper: use FORCE_SCALE when force_scale_pixbuf is set

When force_scale_pixbuf is set, the icon helper will scale the
icon to the requested size (either the pixel size, or the resolved
icon size), so we can just as well instruct the icon theme code
to do the scaling for us.

GtkIconTheme: Drop an unused variable

We no longer need a separate field for symbolic icon size,
now that we are using the directory size.

GtkIconTheme: Load symbolic icons at the same size

Reuse the scale information that we have from loading icons
normally, when loading a symbolic icon, so that we apply the
same size constraints.

This commit assumes that svgs have the nominal size of the
directory they are in, which will be true for all current
symbolic icons.

Adapt icontheme tests

With our new, improved icon theme spec compliance, some of the
tests need updates to their expected outcomes.

GtkIconInfo: Small struct packing improvement

This makes the GtkIconInfo struct a bit smaller.

GtkIconInfo: Drop an unused field

The GtkIconInfo threshold field was never used, so lets not
carry it around.

GtkIconTheme: Respect limits in theme dirs

Previously, we were taking thresholds and min/max sizes into
account when choosing the best theme dir, but when it came
to loading the icon, we always scaled icons from scalable
directories all the way, ignoring the min/max size limits.

This commit changes things around so that we now load icons
in Threshold directories at their nominal size, and scale
icons in Scalable directories only up to the specified limits.

To override this and keep the previous behaviour of scaling
all the way to the desired size, use the GTK_ICON_THEME_FORCE_SIZE
flag.

gdkwindow: Provide a default implementation of process_updates_recurse

As a quick code cleanup.

gdkwindow-broadway: Move the dirty flag setting to end_paint

This is where it belongs -- process_updates_recurse is the wrong place
for it.

gdkwindow: Move a variable closer to where it's used

To make the code clearer.

gdkwindow: Remove an unused variable

GtkSwitch: Implement hover for GtkSwitch

This makes switches more similar to other activatable widgets
like buttons or scales.

gdkwidow: Make queue_antiexpose optional

gdkwindow-win32: Remove queue_antiexpose implementation

Since the Win32 code never actually called InvalidateRgn or used the
Win32 update area at all, that meant the only thing that could possibly
invalidate the window was the Win32 window manager as part of scrolling
or resizing, which would also send it a WM_PAINT message.

But the WM_PAINT handling called BeginPaint / EndPaint, which clears the
update area completely! We also draw out-of-band, not directly when
handling WM_PAINT, so there's no way that the update area inside the
Win32 WM would match our local one.

There is no possible way that this queue_antiexpose implementation could
do anything. Remove it.

gdkevents-win32: Remove dead code

gdkwindow: Don't bother with a return parameter for queue_antiexpose

Standard refcounting works perfectly well. Don't give us the opportunity
for more memory leaks.

gdkwindow-broadway: Squash memory leak

We don't need to hang on to the update_area.

gdkwindow-broadway: Make queue_antiexpose implementation private

gdkwindow-wayland: Attach new buffers and submit damage in end_paint

process_updates_recurse is simply the wrong place for it to be.

gdkwindow-wayland: Rename listener -> frame_listener

Don't pollute the static namespace here.

docs: add missing apostrophe

gdkwindow: Correct a signature

We removed the parameter from callers and from the implementation, but
not from the signature up top. I didn't notice because the branch I was
working on removed the signature entirely.

gdkwindow: Remove an extra region copy

This code is only called with the current paint region as its argument.
Instead of having to copy it and do a no-op intersect against itself,
just use the current paint directly.

gdkwindow: Clean up code

Introduce a temporary to make the code a bit more clear under these cases.

gdkwindow: Clean up formatting

Tabs to spaces, add some whitespace.

gdkwindow: Add some extra documentation to gdk_cairo_create

Mention the need to wrap the function inside
gdk_window_begin_paint_region and gdk_window_end_paint.

widgetpath: Deprecate regions

This is a foolowup to d80bf0790d0903ba2825f306b6f7435529f922e2

gtkwidget: Add a warning about set_double_buffered

As requested on the mailing list.

testwindows: Remove another reference to set_double_buffered

gtktrayicon-x11: Stop using set_double_buffered

There's nothing better we can do for this case, now that we always
redirect drawing to a temporary pixmap. Maybe since this is already
X11-specific code, we should just do everything with Xlib directly.

Adwaita: Fix highlight on check buttons

The .button style class does not apply to check and radio
buttons, and .check only applies to menuitems.

gdkwindow: Squash a leak in the subsurface case

cairo_surface_create_for_rectangle takes a ref on the parent surface,
so we need to drop ours.

Rename get_window_surface to ref_window_surface to make the code more
clear and to stop this error from happening again.

gdkwindow: Don't create a subsurface if we don't need to

In the case we have a native window, we don't need to crop it to a
non-native subrectangle, as abs_x / abs_y / width / height are all 0.

gdkwindow: Remove the ability to call begin_paint_region more than once

Previously, each begin_paint_region added to a stack of current paints,
and when end_paint was called, the paint was popped off of the stack and
the surface was composited into the parent paint surface.

However, the code was broken in the case of a backend like Wayland which
didn't keep track of nested calls and simply wiped and returned the
native impl backing surface every time.

Since this feature is flat out unused by GTK+ and we don't want to
really support tricksy things like these for other clients, just remove
the feature. If somebody does call begin_paint_region more than once,
warn and return without doing anything.

gdkwindow: Remove the internal cairo_surface used for out-of-band painting

Traditionally, the way painting was done in GTK+ was with the
"expose-event" handler, where you'd use GDK methods to do drawing on
your surface. In GTK+ 2.24, we added cairo support with gdk_cairo_create,
so you could paint your graphics with cairo.

Since then, we've added client-side windows, double buffering, the paint
clock, and various other enhancements, and the modern way to do drawing
is to connect to the "draw" signal on GtkWidget, which hands you a
cairo_t. To do double-buffering, the cairo_t we hand you is actually on
a secret surface, not the actual backing store of the window, and when
the draw handler completes we blit it into the main backing store
atomically.

The code to do this is with the APIs gdk_window_begin_paint_region,
which creates the temporary surface, and gdk_window_end_paint which
blits it back into the backing store. GTK+'s implementation of the
"draw" signal uses these APIs.

We've always sort-of supported people calling gdk_cairo_create
"outside" of a begin_paint / end_paint like old times, but then you're
not getting the benefit of double-buffering, and it's harder for GDK to
optimize.

Additionally, newer backends like Mir and Wayland can't actually support
this model, since they're based on double-buffering and swapping buffers
at various points in time. If we hand you a random cairo_t, we have no
idea when is a good time to swap.

Remove support for this.

This is technically a GDK API break: a warning is added in cases where
gdk_cairo_create is called outside of a paint cycle, and the returned
surface is a dummy that won't ever be composited back onto the main
surface. Testing with complex applications like Ardour didn't produce
any warnings.

gtkwidget: Widgets are now always double-buffered

gtk_widget_set_double_buffered is now deprecated, and we don't support
non-double-buffered widgets. This means that under normal circumstances,
paints are never outside of a begin_paint / end_paint sequence, which
natively-double-buffered backends like Wayland can't possibly support.

gdkwindow: Deprecate gdk_window_flush

The function does nothing.

GtkIconTheme: Drop an unused function

GtkIconTheme: Small cleanup

Make the default symbolic colors a little more readable in the code.

GtkIconTheme: Trivial cleanup

Drop the _-prefix from a static function.
We've agreed to no stop using those prefixes.

Avoid a compiler warning

The name_offset variable is indeed unused.

Big spinner

GtkIconTheme: Don't create empty IconThemeDir structs

Starting gtk3-demo currently creates ~1200 of these, most
of them for nonexisting directories. Lets try to avoid that.

GtkIconCache: Add api to find if directories are empty

This will help in not creating structs for tons of empty
directories.

GtkStatusbar: Add some space between the frame and the label

Looks better this way.

Adwaita: check/radio selector simplification

widget factory: Use a property action for the toolbar toggle

This is slightly more elegant than doing it manually.

widget factory: Add a statusbar

A widget that is not represented in the widget factory is likely
to not get proper attention from theme authors.

docs: Drop image references for stock items

Most of those images are no longer in our tree, so just
say which ones have icons and rtl variants, instead of
showing it.

wayland: Prevent stale paints and weird artifacts when using Weston

Weston releases buffers almost immediately after they're done, which
means that GTK+ doesn't use a temporary surface and instead paints
directly onto the SHM backing store that Weston will use.

Normally, after painting to the temporary surface, GTK+ *replaces*
the existing backing surface with CAIRO_OPERATOR_SOURCE. However,
if we immediately paint to the backing surface, it might have junk
from the last paint in it. So clear out the backing surface whenever
somebody calls begin_paint_region().

Maybe we should just always use the temporary surface like the X11
codepath, since that prevents us from having to do weird things like
this, but oh well.

GtkIconTheme: Keep gtk_icon_theme_add_resource_path private for now

We need to discuss some details of the semantics before
we can make this public.

Revert "Document gtk_icon_theme_add_resource_path"

This reverts commit 988b6b483dc9c21d050161d26652bb1006b9c686.

Revert "Deprecate gtk_icon_theme_add_builtin_icon"

This reverts commit 923eeda2d3cae13d02b0a9d2e5774b12eed999e0.

Deprecate gtk_icon_theme_add_builtin_icon

Document gtk_icon_theme_add_resource_path

Adapt the icontheme tests

With the new way of dealing with the legacy stock icons,
the builtin test needed some small updates.

GtkIconTheme: Include legacy stock icons from resources

Add /org/gtk/libgtk/icons as a resource path, and ensure
that we always parse an index.theme file for hicolor which
makes the builtin icons available as part of the hicolor theme.

GtkIconTheme: Allow loading icons from resources

We add a new API, gtk_icon_theme_add_resource_path, which
can be used to add resource path as a base location for
icon theme content, similar to gtk_icon_theme_append_search_path.

GtkIconTheme: Drop the all_icons hash table

It is not used anywhere anymore.

GtkIconTheme: Stop using the all_icons hash table

This hash table was used in gtk_icon_theme_has_icon; we can
implement that function just as well without it and save some
memory.

icontheme: Add some more tests

This test checks that gtk_icon_theme_list_icons and
gtk_icon_theme_has_icon work as expected.

Formatting fixes

GtkIconTheme: Drop the code for parsing .icon files

Modern icon themes don't ship .icon files anyway.

GtkIconTheme: Deprecate functionality related to .icon files

This functionality is not used at all in modern icon themes.

Drop the builtin icon cache

We don't need it anymore, now that the non-standard stock icons
are using resources.

Add non-standard stock icons as resources

This will allow us to remove the builtin icon cache in the future
while keeping the deprecated stock icons working.

Drop all standard icons from the builtin icontheme

We're going to require a complete icon theme, and we have
a test that checks for all the icons we use, so there is
no need to include all these fallback icons.